Skip to content

refactor!: Refactor AlloyDBVectorStore to depend on PGVectorstore #435

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Aug 12, 2025

Conversation

dishaprakash
Copy link
Contributor

@dishaprakash dishaprakash commented Jul 24, 2025

refactor!: Refactor AlloyDBVectorStore to depend on PGVectorstore

AlloyDBAsyncVectorStore and AlloyDBVectorStore are refactored to depend on the AsyncPGVectorStore and PGVectorStore respectively.

Hybrid search is now supported in AlloyDBVectorStore. Tests for hybrid search have been added.

BREAKING CHANGE:
The support for text filters in similiarity searches has been removed, and we support only dictionary filters now.

The dependency on langchain-core and pgvector is based on LangChain-Postgres's specifications.

@dishaprakash dishaprakash requested review from a team as code owners July 24, 2025 12:02
@product-auto-label product-auto-label bot added the api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API. label Jul 24, 2025
@dishaprakash dishaprakash changed the base branch from main to refactor-engine July 24, 2025 12:02
@averikitsch averikitsch merged commit 825fa1c into refactor-engine Aug 12, 2025
8 checks passed
@averikitsch averikitsch deleted the refactor-vs branch August 12, 2025 21:53
averikitsch added a commit that referenced this pull request Aug 13, 2025
… PGVectorstore and PGEngine respectively (#434)

* chore: Refactor AlloyDBEngine to depend on PGEngine

* fix tests

* dependency fux

* dependency fix

* change imports

* change imports

* change imports

* fix breaking imports

* linter fix and add vector index tests

* refactor!: Refactor AlloyDBVectorStore to depend on PGVectorstore (#435)

* refactor!: Refactor AlloyDBVectorStore to depend on PGVectorstore

* Linter fix

* Fix tests

* Fix tests

* fix tests

* linter fix

* fix vectorstore

* add all existing tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix dependecies

* Add tests for hybrid search

* linter fix

* fix test

* Add better documentation for breaking change

* minor fix

* update deps

* Re-expose hybrid search config

* add header

* linter fix

* re-expose hybrid search configs from init

* re-expose hybrid search configs from init

---------

Co-authored-by: Averi Kitsch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants